fix: close browser after export#4373
Conversation
|
@howenyap is attempting to deploy a commit to the modsbot's projects Team on Vercel. A member of the Team first needs to authorize it. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4373 +/- ##
==========================================
+ Coverage 54.52% 56.40% +1.88%
==========================================
Files 274 317 +43
Lines 6076 6962 +886
Branches 1455 1679 +224
==========================================
+ Hits 3313 3927 +614
- Misses 2763 3035 +272 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@greptile |
first greptile scan kinda nervous |
|
b79003c to
2bf0914
Compare
|
@greptile |
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
c023fcf to
07a68ec
Compare
|
@greptileai review |
07a68ec to
ce11372
Compare
|
PR author is not in the allowed authors list. |
Context
The serverless export service opens a browser but only returns the page handler
Then, cleanup only closes the page, leaving the browser handle open.
Implementation
openfunctionOther Information
Disclaimer:
I found this leak with Codex Security and told @ravern about it!